[#750] Writer Stats: rename to Donations Received + add Royalties Claimed#753
[#750] Writer Stats: rename to Donations Received + add Royalties Claimed#753realproject7 merged 1 commit intomainfrom
Conversation
…Claimed' 1. Rename "Received" label to "Donations Received" in Writer Stats 2. Add "Royalties Claimed" row using existing claimedRoyalties data from the profile page (on-chain getRoyaltyInfo) 3. Only shows Royalties Claimed when amount > 0 Fixes #750 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub. |
project7-interns
left a comment
project7-interns
left a comment
There was a problem hiding this comment.
T2b APPROVE
Clean, minimal change. Label rename is clear, royalties row follows the exact same pattern as Donations Received (styling, conditional rendering, USD conversion).
Minor note (pre-existing, not blocking): formatUnits(..., 18) hardcodes decimals — applies to the existing totalDonations line too, not just this PR.
LGTM.
project7-interns
left a comment
project7-interns
left a comment
There was a problem hiding this comment.
Verdict: APPROVE
Summary
This PR cleanly renames the Writer Stats label to "Donations Received" and adds a "Royalties Claimed" row using the existing profile-level claimedRoyalties data, which matches issue #750 with a minimal one-file change.
Findings
- None.
Decision
Approving because the change stays within the ticket scope, follows the existing Writer Stats pattern, and the current checks are passing.
2 participants
Summary
claimedRoyaltiesdata from profile page (on-chaingetRoyaltyInfo)claimedRoyaltiesfrom parent ProfilePage to StoriesTabFixes #750
Test plan
npm run buildpasses🤖 Generated with Claude Code